Skip to content

fix(ci): check PR author not actor for bot linear exemption#25

Merged
luke-ht merged 1 commit into
mainfrom
fix/linear-check-bot-pr-author
May 4, 2026
Merged

fix(ci): check PR author not actor for bot linear exemption#25
luke-ht merged 1 commit into
mainfrom
fix/linear-check-bot-pr-author

Conversation

@luke-ht

@luke-ht luke-ht commented May 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • github.actor = whoever triggered the workflow (last pusher) — changes when a human commits to a bot branch
  • github.event.pull_request.user.login = PR creator — immutable
  • Renovate PR #336 in devops has been blocked for 2 months because a manual push changed the actor from renovate[bot] to a human username, bypassing the exemption

Switching to PR_AUTHOR fixes the exemption for all repos using this shared workflow, regardless of whether someone manually pushes to a bot-created branch.

Test plan

  • Merge this to main
  • Sync PR #336 in devopslinear-check should pass/skip for renovate[bot] PR

🤖 Generated with Claude Code


📝 Auto-filled

github.actor reflects whoever triggered the workflow (last pusher),
not the PR creator. If a human pushes a commit to a Renovate branch,
actor becomes their username and the bot exemption is skipped.

Use github.event.pull_request.user.login instead — this is the PR
creator and is immutable regardless of subsequent pushes.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@luke-ht luke-ht merged commit 4575e35 into main May 4, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant